Skip to content

flush logs in the end no matter what #761

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

highker
Copy link

@highker highker commented Aug 5, 2025

Summary:
make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Aug 5, 2025
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D79620358

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D79620358

highker pushed a commit to highker/monarch that referenced this pull request Aug 5, 2025
Summary:
Pull Request resolved: meta-pytorch#761

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D79620358

highker pushed a commit to highker/monarch that referenced this pull request Aug 5, 2025
Summary:
Pull Request resolved: meta-pytorch#761

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
highker pushed a commit to highker/monarch that referenced this pull request Aug 5, 2025
Summary:

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
@highker highker force-pushed the export-D79620358 branch 2 times, most recently from 4cb03a5 to 8390893 Compare August 5, 2025 16:52
highker pushed a commit to highker/monarch that referenced this pull request Aug 5, 2025
Summary:

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D79620358

highker pushed a commit to highker/monarch that referenced this pull request Aug 5, 2025
Summary:
Pull Request resolved: meta-pytorch#761

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D79620358

highker pushed a commit to highker/monarch that referenced this pull request Aug 5, 2025
Summary:
Pull Request resolved: meta-pytorch#761

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
highker pushed a commit to highker/monarch that referenced this pull request Aug 6, 2025
Summary:

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
highker pushed a commit to highker/monarch that referenced this pull request Aug 6, 2025
Summary:

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D79620358

highker pushed a commit to highker/monarch that referenced this pull request Aug 6, 2025
Summary:
Pull Request resolved: pytorch-labs#761

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D79620358

highker pushed a commit to highker/monarch that referenced this pull request Aug 6, 2025
Summary:
Pull Request resolved: pytorch-labs#761

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
@highker highker force-pushed the export-D79620358 branch 2 times, most recently from 7d9e8bb to c4b7514 Compare August 6, 2025 06:07
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D79620358

highker pushed a commit to highker/monarch that referenced this pull request Aug 6, 2025
Summary:
Pull Request resolved: meta-pytorch#761

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
highker pushed a commit to highker/monarch that referenced this pull request Aug 6, 2025
Summary:

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
@highker highker force-pushed the export-D79620358 branch 2 times, most recently from ace056f to 2e66d57 Compare August 6, 2025 08:01
highker pushed a commit to highker/monarch that referenced this pull request Aug 6, 2025
Summary:

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D79620358

highker pushed a commit to highker/monarch that referenced this pull request Aug 6, 2025
Summary:
Pull Request resolved: meta-pytorch#761

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D79620358

highker pushed a commit to highker/monarch that referenced this pull request Aug 6, 2025
Summary:
Pull Request resolved: meta-pytorch#761

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D79620358

highker pushed a commit to highker/monarch that referenced this pull request Aug 6, 2025
Summary:
Pull Request resolved: meta-pytorch#761

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
@highker highker force-pushed the export-D79620358 branch 2 times, most recently from 1edc19e to 75d3cd5 Compare August 7, 2025 05:45
highker pushed a commit to highker/monarch that referenced this pull request Aug 7, 2025
Summary:
X-link: meta-pytorch#761

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
highker pushed a commit to highker/monarch that referenced this pull request Aug 7, 2025
Summary:
X-link: meta-pytorch#761

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D79620358

highker pushed a commit to highker/monarch that referenced this pull request Aug 7, 2025
Summary:
X-link: meta-pytorch#761

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
Summary:
Pull Request resolved: meta-pytorch#761

X-link: meta-pytorch#761

make sure we are able to flush whatever remaining when program ends.
However, we could still miss logs as log through channel is not always
guaranteed to be streamed back when a program ends.

Differential Revision: D79620358
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D79620358

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 7a77021.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Meta Open Source bot. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants